Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

feat(all): use an unified transaction sender implementation #560

Merged
merged 34 commits into from
Feb 25, 2024
Merged

Conversation

mask-pp
Copy link
Contributor

@mask-pp mask-pp commented Feb 21, 2024

No description provided.

@davidtaikocha davidtaikocha linked an issue Feb 21, 2024 that may be closed by this pull request
@davidtaikocha davidtaikocha changed the title feat(sender): Sender feat(all): use an unified transaction sender implementation Feb 21, 2024
Copy link

codecov bot commented Feb 22, 2024

Codecov Report

Attention: Patch coverage is 52.84360% with 199 lines in your changes are missing coverage. Please review.

Project coverage is 53.12%. Comparing base (6d6d9c6) to head (f255da3).
Report is 1 commits behind head on main.

Files Patch % Lines
internal/sender/common.go 40.13% 86 Missing and 5 partials ⚠️
internal/sender/sender.go 62.61% 66 Missing and 17 partials ⚠️
proposer/proposer.go 55.26% 13 Missing and 4 partials ⚠️
pkg/rpc/utils.go 20.00% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #560      +/-   ##
==========================================
+ Coverage   52.86%   53.12%   +0.25%     
==========================================
  Files          45       47       +2     
  Lines        5769     6080     +311     
==========================================
+ Hits         3050     3230     +180     
- Misses       2384     2498     +114     
- Partials      335      352      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@davidtaikocha davidtaikocha merged commit 1bd56c0 into main Feb 25, 2024
4 checks passed
@davidtaikocha davidtaikocha deleted the sender branch February 25, 2024 05:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use an unified Transaction Sender module
2 participants